Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade ScanCode-toolkit to version 30.0.1 #330

Merged
merged 7 commits into from
Sep 27, 2021
Merged

Upgrade ScanCode-toolkit to version 30.0.1 #330

merged 7 commits into from
Sep 27, 2021

Conversation

tdruez
Copy link
Contributor

@tdruez tdruez commented Sep 24, 2021

No description provided.

Signed-off-by: Thomas Druez <tdruez@nexb.com>
Signed-off-by: Thomas Druez <tdruez@nexb.com>
Signed-off-by: Thomas Druez <tdruez@nexb.com>
Signed-off-by: Thomas Druez <tdruez@nexb.com>
]
}
]
"packages": []
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@pombredanne any reasons why this package is not detected anymore?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is in the "summary" section, and seems similar to these changes aboutcode-org/scancode-toolkit@b6dd92f that were made for aboutcode-org/scancode-toolkit#1759 as a follow up to PR aboutcode-org/scancode-toolkit#1864

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is a serious regression.

pombredanne and others added 3 commits September 25, 2021 14:05
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Philippe Ombredanne <pombredanne@nexb.com>
Signed-off-by: Thomas Druez <tdruez@nexb.com>
@tdruez tdruez merged commit eb45867 into main Sep 27, 2021
@tdruez tdruez deleted the toolkit-upgrade branch September 27, 2021 05:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants